Skip to content

Add InfraStoragePolicyInfo CRD#3968

Open
skogta wants to merge 1 commit intokubernetes-sigs:masterfrom
skogta:topic/skogta/AddClusterSPICRD
Open

Add InfraStoragePolicyInfo CRD#3968
skogta wants to merge 1 commit intokubernetes-sigs:masterfrom
skogta:topic/skogta/AddClusterSPICRD

Conversation

@skogta
Copy link
Copy Markdown
Contributor

@skogta skogta commented Apr 14, 2026 •
edited
Loading

InfraStoragePolicyInfoSpec defines the desired state of InfraStoragePolicyInfo. It is used to store the information about the storage policy that is exposed to the admin at a cluster level. Currently, it only contains topology information.

CR example:

apiVersion: cns.vmware.com/v1alpha1
kind: InfraStoragePolicyInfo
metadata:
  name: vsan-default-policy
status:
  topology:
    accessibleZones:
      - zone-west-1
      - zone-west-2
  error: ""

Testing done:

Applied CRD manually and ensured that it is getting created.

root@4212ef7b3c931047671ef485ba899967 [ ~ ]# k get infrastoragepolicyinfoes.cns.vmware.com
No resources found
root@4212ef7b3c931047671ef485ba899967 [ ~ ]# k get infraspi
No resources found

WCP (passed): https://jenkins-vcf-csifvt.devops.broadcom.net/job/wcp-instapp-e2e-pre-checkin/1239/
VKS (passed): https://jenkins-vcf-csifvt.devops.broadcom.net/job/vks-instapp-e2e-pre-checkin/1011/

@k8s-ci-robot
Copy link
Copy Markdown
Contributor

k8s-ci-robot commented Apr 14, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: skogta
Once this PR has been reviewed and has the lgtm label, please assign kolluria for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Apr 14, 2026
@skogta skogta closed this Apr 14, 2026
@skogta skogta reopened this Apr 14, 2026
@skogta skogta marked this pull request as draft April 14, 2026 13:28
@k8s-ci-robot k8s-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Apr 14, 2026
@skogta skogta force-pushed the topic/skogta/AddClusterSPICRD branch from 1037f95 to 13d7da2 Compare April 15, 2026 11:24
@skogta skogta changed the title Add ClusterSPI CRD Add InfraStoragePolicyInfo CRD Apr 15, 2026
@skogta skogta force-pushed the topic/skogta/AddClusterSPICRD branch from 13d7da2 to 0678e42 Compare April 15, 2026 11:28
@skogta skogta marked this pull request as ready for review April 15, 2026 11:34
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Apr 15, 2026
@k8s-ci-robot k8s-ci-robot requested a review from gnufied April 15, 2026 11:34
@skogta skogta force-pushed the topic/skogta/AddClusterSPICRD branch from 0678e42 to 0307e43 Compare April 15, 2026 11:35
@deepakkinni
Copy link
Copy Markdown
Collaborator

deepakkinni commented Apr 16, 2026

Triggering CSI-WCP Pre-checkin Pipeline for this PR... Job takes approximately an hour to complete
Jenkins Build #1221

@deepakkinni
Copy link
Copy Markdown
Collaborator

deepakkinni commented Apr 16, 2026

Triggering CSI-TKG Pre-checkin Pipeline for this PR... Job takes approximately an hour to complete
Jenkins Build #1002

@deepakkinni
Copy link
Copy Markdown
Collaborator

deepakkinni commented Apr 16, 2026

FAILED --- Jenkins Build #1221

@deepakkinni
Copy link
Copy Markdown
Collaborator

deepakkinni commented Apr 16, 2026

FAILED --- Jenkins Build #1002

@deepakkinni
Copy link
Copy Markdown
Collaborator

deepakkinni commented Apr 17, 2026

Triggering CSI-TKG Pre-checkin Pipeline for this PR... Job takes approximately an hour to complete
Jenkins Build #1011

@deepakkinni
Copy link
Copy Markdown
Collaborator

deepakkinni commented Apr 17, 2026

Triggering CSI-WCP Pre-checkin Pipeline for this PR... Job takes approximately an hour to complete
Jenkins Build #1235

@deepakkinni
Copy link
Copy Markdown
Collaborator

deepakkinni commented Apr 17, 2026

FAILED --- Jenkins Build #1235

@deepakkinni
Copy link
Copy Markdown
Collaborator

deepakkinni commented Apr 17, 2026

SUCCESS --- Jenkins Build #1011

@deepakkinni
Copy link
Copy Markdown
Collaborator

deepakkinni commented Apr 17, 2026

Triggering CSI-WCP Pre-checkin Pipeline for this PR... Job takes approximately an hour to complete
Jenkins Build #1239

@deepakkinni
Copy link
Copy Markdown
Collaborator

deepakkinni commented Apr 17, 2026

SUCCESS --- Jenkins Build #1239

xing-yang
xing-yang reviewed Apr 19, 2026
View reviewed changes
Comment thread pkg/apis/cnsoperator/infrastoragepolicyinfo/v1alpha1/infrastoragepolicyinfo_types.go Outdated
@skogta skogta force-pushed the topic/skogta/AddClusterSPICRD branch from 0307e43 to c38f01e Compare April 20, 2026 08:22
xing-yang
xing-yang reviewed Apr 21, 2026
View reviewed changes
Comment thread pkg/apis/cnsoperator/infrastoragepolicyinfo/v1alpha1/infrastoragepolicyinfo_types.go Outdated
Comment thread pkg/apis/cnsoperator/infrastoragepolicyinfo/v1alpha1/infrastoragepolicyinfo_types.go Outdated
@skogta skogta force-pushed the topic/skogta/AddClusterSPICRD branch from c38f01e to 032f136 Compare April 22, 2026 04:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@xing-yang xing-yang xing-yang left review comments

@chethanv28 chethanv28 Awaiting requested review from chethanv28

@deepakkinni deepakkinni Awaiting requested review from deepakkinni

@gnufied gnufied Awaiting requested review from gnufied

Assignees

No one assigned

Labels

cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@skogta @k8s-ci-robot @deepakkinni @xing-yang